feat(persona-registry): load a repo's personas from any subdirectory of it - #319

Check if this issue is valid — if so, understand the root cause and fix it. At packages/persona-registry/src/local-personas.ts, line 432:

<comment>When the repository persona directory is already listed in `config.personaDirs`, this adds it a second time before the configured sources and changes precedence. Filter automatic repo entries whose directories are already configured so the explicit position remains authoritative.</comment>

<file context>
@@ -379,6 +429,7 @@ export function buildPersonaSourceDirectories(
       configurable: false,
       nested: true
     },
+    ...repoSourceDirectories(cwd),
     ...config.personaDirs.map((dir, idx) => ({
       source: sourceForPersonaDir(dir, idx, config.userPersonaDir),
</file context>

Check if this issue is valid — if so, understand the root cause and fix it. At packages/persona-registry/src/local-personas.ts, line 233:

<comment>When `$HOME` itself contains `.git` and `cwd` is a child without a nearer repository, `findRepoRoot` returns `$HOME` before honoring the home boundary. This duplicates and relabels personal personas as the higher-priority `repo` layer, so check the home boundary before accepting `.git`.</comment>

<file context>
@@ -212,6 +217,27 @@ export function defaultCwdAgentDir(cwd: string): string {
+  const home = resolvePath(homedir());
+  let dir = resolvePath(cwd);
+  while (true) {
+    if (existsSync(join(dir, '.git'))) return dir;
+    if (dir === home) return undefined;
+    const parent = dirname(dir);
</file context>

Check if this issue is valid — if so, understand the root cause and fix it. At packages/persona-registry/src/index.test.ts, line 183:

<comment>The `findRepoRoot(loose) === undefined` assertion assumes no directory between the temp dir and the walk's stop point is a git repository. `findRepoRoot` checks `.git` at every ancestor until it reaches `homedir()`, so if a runner overrides `TMPDIR` to point inside a checked-out repo (or beneath a git-initialized home for dotfiles), the walk returns that ancestor instead of `undefined` and this test fails spuriously. Making the assertion robust to the ambient temp dir would prevent an environment-dependent failure.</comment>

<file context>
@@ -105,3 +112,78 @@ test('unknown names fail with a typed resolution error', () => {
+  mkdirSync(loose, { recursive: true });
+
+  try {
+    assert.equal(findRepoRoot(loose), undefined);
+    const dirs = buildPersonaSourceDirectories({ cwd: loose, personaDirs: [] }).directories;
+    assert.equal(dirs.some((d) => String(d.source).startsWith('repo')), false);
</file context>